feat: add MLT decoding support by CommanderStorm · Pull Request #2512 · maplibre/martin

CommanderStorm · 2026-01-19T15:18:10Z

No description provided.

martin-tile-utils/src/lib.rs

This PR is one of those easy wins that are nessesary for #2512 but actually not related. Currently, the justfile is not blessing every crate when it should. After this PR, it is. --------- Co-authored-by: Yuri Astrakhan <yuriastrakhan@gmail.com>

Copilot

Pull request overview

Copilot reviewed 14 out of 15 changed files in this pull request and generated 9 comments.

martin-tile-utils/src/lib.rs

tests/expected/martin-cp/flat-with-hash_metadata.txt

martin-tile-utils/src/lib.rs

martin/tests/mb_server_test.rs

Copilot · 2026-01-26T15:11:43Z

tests/fixtures/mbtiles/mlt.sql

+INSERT INTO metadata VALUES('attribution','<a href="https://www.openmaptiles.org/" target="_blank">&copy; OpenMapTiles</a> <a href="https://www.openstreetmap.org/copyright" target="_blank">&copy; OpenStreetMap contributors</a>');
+INSERT INTO metadata VALUES('version','3.15.0');
+INSERT INTO metadata VALUES('type','baselayer');
+INSERT INTO metadata VALUES('format','application/vnd.maplibre-vector-tile');


The 'format' metadata value is set to 'application/vnd.maplibre-vector-tile', which is a content-type string rather than a format identifier. According to Format::parse() implementation, this won't be recognized and will fall back to tile detection. The format should be 'mlt' instead to be consistent with other format values in the metadata table (as defined by Format::metadata_format_value()). While the current approach works because MLT is detectable, it's inconsistent with the MBTiles metadata conventions used elsewhere in the codebase.

This is what planetiler sets.

The value here is not currently specified.

martin-tile-utils/src/lib.rs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

nyurik

looks good, but please rename the verbose mapbox_vector_tile and similar back to mvt

martin-tile-utils/src/lib.rs

nyurik · 2026-01-26T23:14:46Z

martin/tests/mb_server_test.rs

-    let raw_mvt_script = include_str!("../../tests/fixtures/mbtiles/uncompressed_mvt.sql");
-    let (raw_mvt_mbt, raw_mvt_conn, raw_mvt_file) =
-        temp_named_mbtiles(&format!("{test_name}_raw_mvt"), raw_mvt_script).await;
+    let mapbox_vector_tiles_script = include_str!("../../tests/fixtures/mbtiles/world_cities.sql");


lets keep mvt everywhere. Verbosity kills readability

tests/fixtures/mbtiles/mlt.sql

## 🤖 New release * `martin-tile-utils`: 0.6.8 -> 0.6.9 (✓ API compatible changes) * `mbtiles`: 0.15.0 -> 0.15.1 (✓ API compatible changes) * `martin-core`: 0.2.5 -> 0.2.6 (✓ API compatible changes) * `martin`: 1.2.0 -> 1.3.0 <details><summary>Changelog</summary> ## `mbtiles` <blockquote> ## [0.15.1](mbtiles-v0.15.0...mbtiles-v0.15.1) - 2026-01-27 ### Added - add MLT decoding support ([#2512](#2512)) - migrate our log library to tracing ([#2494](#2494)) ### Other - unignore `diff_and_patch_bsdiff` test with unique SQLite database names ([#2480](#2480)) - *(mbtiles)* remove the prefix-ism around how files are named for binary diff copy and simpify their naming ([#2478](#2478)) - *(mbtiles)* add assertion messages what we are checking to the copy tests ([#2477](#2477)) </blockquote> ## `martin-core` <blockquote> ## [0.2.6](martin-core-v0.2.5...martin-core-v0.2.6) - 2026-01-27 ### Added - migrate our log library to tracing ([#2494](#2494)) - *(martin-core)* Allow glyph ranges more than 0xFFFF ([#2438](#2438)) ### Fixed - *(perf)* Remove FontSources.masks as they were consuming large amounts of memory even when no font sources were set ([#2519](#2519)) - improve error message if no SVG sprite files are present ([#2516](#2516)) ### Other - move our imports to tracing ([#2500](#2500)) - *(deps)* shear our dependencys ([#2497](#2497)) </blockquote> ## `martin` <blockquote> ## [1.3.0](martin-v1.2.0...martin-v1.3.0) - 2026-01-27 ### Added - *(srv)* Add `route_prefix` configuration for native subpath support without the need of a reverse proxy override ([#2523](#2523)) - add MLT decoding support ([#2512](#2512)) - migrate our log library to tracing ([#2494](#2494)) - improve martin-cp progress output time estimate ([#2491](#2491)) - *(pg)* include ID column info for tables ([#2485](#2485)) - *(pg)* support PostgreSQL materialized views ([#2279](#2279)) - *(martin-core)* Allow glyph ranges more than 0xFFFF ([#2438](#2438)) ### Fixed - *(ui)* clipboard copy for http://0.0.0.0:3000 and unify implementations ([#2487](#2487)) - the `Copy` icon displaying nicely, next to the text and with enough padding ot all items ([#2483](#2483)) - update copy text to include icon for better visibility ([#2482](#2482)) - *(perf)* Remove FontSources.masks as they were consuming large amounts of memory even when no font sources were set ([#2519](#2519)) - improve error message if no SVG sprite files are present ([#2516](#2516)) ### Other - move our request logging to tracing ([#2508](#2508)) - move our imports to tracing ([#2500](#2500)) - *(deps)* shear our dependencys ([#2497](#2497)) - *(ui)* adjust margin for copy icon in URL component ([#2489](#2489)) - unignore `diff_and_patch_bsdiff` test with unique SQLite database names ([#2480](#2480)) - *(mbtiles)* remove the prefix-ism around how files are named for binary diff copy and simpify their naming ([#2478](#2478)) - *(mbtiles)* add assertion messages what we are checking to the copy tests ([#2477](#2477)) </blockquote> </details> --- This PR was generated with [release-plz](https://github.com/release-plz/release-plz/). --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

add mlt support

6cca2c7

CommanderStorm commented Jan 19, 2026

View reviewed changes

martin-tile-utils/src/lib.rs Outdated Show resolved Hide resolved

Apply suggestion from @CommanderStorm

f888876

CommanderStorm changed the title ~~chore: add mlt support~~ feat: add mlt support Jan 19, 2026

CommanderStorm added the bless label Jan 19, 2026

github-actions bot removed the bless label Jan 19, 2026

CommanderStorm added 3 commits January 19, 2026 17:19

fix naming issues

78360a1

fix clippy issues

eebc2bd

make mock_sources error better to understand

87b1022

CommanderStorm added the bless label Jan 19, 2026

github-actions bot removed the bless label Jan 19, 2026

chore(fmt): apply pre-commit formatting fixes

0654358

CommanderStorm added the bless label Jan 19, 2026

github-actions bot removed the bless label Jan 19, 2026

CommanderStorm added 2 commits January 19, 2026 17:26

make the naming make a bit more sense

ba9f1e3

add aproper mlt test

d04e295

CommanderStorm added the bless label Jan 19, 2026

github-actions bot removed the bless label Jan 19, 2026

CommanderStorm and others added 8 commits January 19, 2026 18:31

made the formatting nicer

34a0298

make the detection a bit less crude

6945953

chore(fmt): apply pre-commit formatting fixes

c3c4041

improve the detection logic

8d19d5a

update docs

4180676

implement proper format detection

051c35c

clippy fixes

99e4b69

apply change upstream

84e9f61

CommanderStorm changed the title ~~feat: add mlt support~~ feat: add MLT decoding support Jan 19, 2026

CommanderStorm requested a review from nyurik January 19, 2026 21:57

CommanderStorm added the bless label Jan 19, 2026

github-actions bot removed the bless label Jan 19, 2026

CommanderStorm and others added 4 commits January 25, 2026 20:32

make the decoding for mlt more explicit

fe6c796

chore(fmt): apply pre-commit formatting fixes

980c610

Merge branch 'main' into mlt-support

6a59755

fix clippy lint

f556edf

CommanderStorm requested a review from Copilot January 26, 2026 15:05

Copilot started reviewing on behalf of CommanderStorm January 26, 2026 15:06 View session

Copilot AI reviewed Jan 26, 2026

View reviewed changes

CommanderStorm and others added 3 commits January 26, 2026 16:30

Update martin-tile-utils/src/lib.rs

4657dd2

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update martin-tile-utils/src/lib.rs

13330a6

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update martin-tile-utils/src/lib.rs

572c52b

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

nyurik approved these changes Jan 26, 2026

View reviewed changes

CommanderStorm and others added 4 commits January 27, 2026 01:37

make a few things more ideomatic

a8fd76e

apply requested renaming

281459b

fix clippy linsts

4536583

chore(fmt): apply pre-commit formatting fixes

d5b731d

CommanderStorm commented Jan 27, 2026

View reviewed changes

tests/fixtures/mbtiles/mlt.sql Outdated Show resolved Hide resolved

Apply suggestion from @CommanderStorm

3d02db1

CommanderStorm enabled auto-merge (squash) January 27, 2026 00:47

CommanderStorm added the bless label Jan 27, 2026

github-actions bot removed the bless label Jan 27, 2026

fix the way blessing works again

8853666

CommanderStorm added the bless label Jan 27, 2026

github-actions bot removed the bless label Jan 27, 2026

CommanderStorm added 2 commits January 27, 2026 03:58

bless the output

0c13ffe

fix another blessing issue

96d422b

CommanderStorm merged commit 1c1c7fe into maplibre:main Jan 27, 2026
40 checks passed

CommanderStorm mentioned this pull request Jan 27, 2026

chore: release #2481

Merged

vinayakkulkarni mentioned this pull request Feb 18, 2026

feat: MapLibre Tiles (MLT) format support vinayakkulkarni/tileserver-rs#609

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add MLT decoding support#2512

feat: add MLT decoding support#2512
CommanderStorm merged 42 commits intomaplibre:mainfrom
CommanderStorm:mlt-support

CommanderStorm commented Jan 19, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

CommanderStorm Jan 26, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nyurik left a comment

Uh oh!

Uh oh!

nyurik Jan 26, 2026

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

CommanderStorm commented Jan 19, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

CommanderStorm Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nyurik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nyurik Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants